* Fix QA group dispatch: pass QA via the `qa` kwarg, not `groups`

The canonicalization conversion (#154) placed the QA group inside the
`groups` Dict, but SciMLTesting's `run_tests` routes `GROUP="QA"` through
a dedicated `elseif group == "QA"` branch that consumes the `qa` keyword
argument and short-circuits before the `groups` table is ever consulted.
With `qa` unset, that branch throws

    ArgumentError: run_tests: GROUP="QA" was requested but no `qa` body
    was provided

so the QA CI job failed on master before loading FEniCS. Moving the QA
spec to the `qa` kwarg (matching the documented SciMLTesting form) makes
`GROUP="QA"` reach the QA body, while `all = ["Core"]` keeps QA out of the
default `GROUP="All"`/local `]test` run as before.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* ci: re-trigger against SciML/.github v1.15.1

Verify the scoped GitHub-hosted runner routing (SciML/.github #97) now
sends this repo's apt-packages/container legs to ubuntu-24.04.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* Fix FEniCS package-quality (QA) failures surfaced by the QA group

The QA group now actually runs FEniCS's own Aqua/JET checks, surfacing
six pre-existing package-quality issues. Fix the real causes:

- Type piracy + Base.write method ambiguity: `write(path, solution,
  time::Number)` extended `Base.write` with no FEniCS-owned argument
  type, which is piracy and also clashed with `Base.write(::IO, x1,
  xs...)`. Stop importing `write` from Base so `write`/`store` are
  FEniCS-owned functions (still exported); behavior unchanged.
- Undefined export: `export fenicsclass` referenced a nonexistent
  binding; the macro is `@fenicsclass`. Export the macro.
- JET no-matching-method: `Argument(V, number)` lowered to
  `Argument(::Any, ::Any, ::Nothing)` because `part::StringOrSymbol`
  cannot accept its own `nothing` default. Widen to
  `Union{StringOrSymbol, Nothing}`.
- Stale dependency: ProgressMeter was declared but unused (only a
  dead `@require ... using ProgressMeter` no-op). Drop it from deps,
  compat, and the no-op __init__ block.
- deps_compat: add a `[compat]` entry for the LinearAlgebra stdlib.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* Keep write/store as Base.write extension; fix piracy+ambiguity via arg types

The prior commit removed `write` from the `Base` import to kill the type
piracy, but that made `FEniCS.write` a distinct exported binding that
clashed with the exported `Base.write` ("both FEniCS and Base export
write"), breaking the Core examples (tutorial8/9: `UndefVarError: write`).

Restore `import Base: ...write` so `write` stays a single function, and
fix the two QA findings on it through argument types instead:

- Type piracy: `write` now takes `solution::fenicsobject` (a FEniCS-owned
  type), so the method is owned by FEniCS rather than pirating Base. The
  unused, unavoidably-piratical `write(::Any, ::PyObject, ::Number)`
  overload is dropped (use `store` for raw vectors like `vector(u)`).
- Method ambiguity: type `path::PyObject` (what `XDMFFile`/`TimeSeries`
  return), which is disjoint from `Base.write(::IO, ...)` and
  `write(::AbstractString, ...)`, removing both ambiguities.

`store` is FEniCS-owned (not a Base function), so its overloads are not
piracy; `path::PyObject` is added there too for symmetry/correctness.

Co-Authored-By: Chris Rackauckas <accounts@chrisrackauckas.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: ChrisRackauckas-Claude <accounts@chrisrackauckas.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>